Various reentrancy fixes for widgets being destroyed out of

Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>

        Various reentrancy fixes for widgets being destroyed
        out of focus-out-event. (#128821, Grant Gayed)

        * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default):
        Ref window and widget over callbacks.

        * gtk/gtkwidget.c (gtk_widget_hide)
          gtk/gtkwidget.c (gtk_widget_set_child_visible):
        Ref the widget before calling _gtk_window_unset_focus_and_default(),
        since that can call user callbacks.

        * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
        Do nothing if the widget isn't realized.

diff --git a/ChangeLog b/ChangeLog
index 5dff67430965bf93164514e2985ea9344b0d96c5..f6c17e4a98b7fd5409762e12a7a49749a7e7206b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>
+
+        Various reentrancy fixes for widgets being destroyed
+       out of focus-out-event. (#128821, Grant Gayed)
+
+       * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default): 
+       Ref window and widget over callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_hide)
+         gtk/gtkwidget.c (gtk_widget_set_child_visible):
+       Ref the widget before calling _gtk_window_unset_focus_and_default(),
+       since that can call user callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
+       Do nothing if the widget isn't realized.
+
 Fri Feb 13 01:31:44 2004  Matthias Clasen  <maclas@gmx.de>
 
        * gtk/gtkbutton.c (_gtk_button_paint): Don't draw bevels around

diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index 5dff67430965bf93164514e2985ea9344b0d96c5..f6c17e4a98b7fd5409762e12a7a49749a7e7206b 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,19 @@
+Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>
+
+        Various reentrancy fixes for widgets being destroyed
+       out of focus-out-event. (#128821, Grant Gayed)
+
+       * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default): 
+       Ref window and widget over callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_hide)
+         gtk/gtkwidget.c (gtk_widget_set_child_visible):
+       Ref the widget before calling _gtk_window_unset_focus_and_default(),
+       since that can call user callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
+       Do nothing if the widget isn't realized.
+
 Fri Feb 13 01:31:44 2004  Matthias Clasen  <maclas@gmx.de>
 
        * gtk/gtkbutton.c (_gtk_button_paint): Don't draw bevels around

diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4
index 5dff67430965bf93164514e2985ea9344b0d96c5..f6c17e4a98b7fd5409762e12a7a49749a7e7206b 100644 (file)
--- a/ChangeLog.pre-2-4
+++ b/ChangeLog.pre-2-4
@@ -1,3 +1,19 @@
+Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>
+
+        Various reentrancy fixes for widgets being destroyed
+       out of focus-out-event. (#128821, Grant Gayed)
+
+       * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default): 
+       Ref window and widget over callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_hide)
+         gtk/gtkwidget.c (gtk_widget_set_child_visible):
+       Ref the widget before calling _gtk_window_unset_focus_and_default(),
+       since that can call user callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
+       Do nothing if the widget isn't realized.
+
 Fri Feb 13 01:31:44 2004  Matthias Clasen  <maclas@gmx.de>
 
        * gtk/gtkbutton.c (_gtk_button_paint): Don't draw bevels around

diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6
index 5dff67430965bf93164514e2985ea9344b0d96c5..f6c17e4a98b7fd5409762e12a7a49749a7e7206b 100644 (file)
--- a/ChangeLog.pre-2-6
+++ b/ChangeLog.pre-2-6
@@ -1,3 +1,19 @@
+Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>
+
+        Various reentrancy fixes for widgets being destroyed
+       out of focus-out-event. (#128821, Grant Gayed)
+
+       * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default): 
+       Ref window and widget over callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_hide)
+         gtk/gtkwidget.c (gtk_widget_set_child_visible):
+       Ref the widget before calling _gtk_window_unset_focus_and_default(),
+       since that can call user callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
+       Do nothing if the widget isn't realized.
+
 Fri Feb 13 01:31:44 2004  Matthias Clasen  <maclas@gmx.de>
 
        * gtk/gtkbutton.c (_gtk_button_paint): Don't draw bevels around

diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8
index 5dff67430965bf93164514e2985ea9344b0d96c5..f6c17e4a98b7fd5409762e12a7a49749a7e7206b 100644 (file)
--- a/ChangeLog.pre-2-8
+++ b/ChangeLog.pre-2-8
@@ -1,3 +1,19 @@
+Thu Feb 12 21:58:20 2004  Owen Taylor  <otaylor@redhat.com>
+
+        Various reentrancy fixes for widgets being destroyed
+       out of focus-out-event. (#128821, Grant Gayed)
+
+       * gtk/gtkwindow.c (_gtk_window_unset_focus_and_default): 
+       Ref window and widget over callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_hide)
+         gtk/gtkwidget.c (gtk_widget_set_child_visible):
+       Ref the widget before calling _gtk_window_unset_focus_and_default(),
+       since that can call user callbacks.
+
+       * gtk/gtkwidget.c (gtk_widget_invalidate_widget_windows):
+       Do nothing if the widget isn't realized.
+
 Fri Feb 13 01:31:44 2004  Matthias Clasen  <maclas@gmx.de>
 
        * gtk/gtkbutton.c (_gtk_button_paint): Don't draw bevels around

diff --git a/gtk/gtkwidget.c b/gtk/gtkwidget.c
index f43e9943450c4bb0fb7310a7d09cbe0f6c8678c9..874a3d13604b5a1011ed6ed4e535e996e7fc7ae8 100644 (file)
--- a/gtk/gtkwidget.c
+++ b/gtk/gtkwidget.c
@@ -2040,10 +2040,11 @@ gtk_widget_hide (GtkWidget *widget)
   if (GTK_WIDGET_VISIBLE (widget))
     {
       GtkWidget *toplevel = gtk_widget_get_toplevel (widget);
+      
+      g_object_ref (widget);
       if (toplevel != widget && GTK_WIDGET_TOPLEVEL (toplevel))
        _gtk_window_unset_focus_and_default (GTK_WINDOW (toplevel), widget);
 
-      g_object_ref (widget);
       g_signal_emit (widget, widget_signals[HIDE], 0);
       if (!GTK_WIDGET_TOPLEVEL (widget))
        gtk_widget_queue_resize (widget);
@@ -2613,6 +2614,9 @@ static void
 gtk_widget_invalidate_widget_windows (GtkWidget *widget,
                                      GdkRegion *region)
 {
+  if (!GTK_WIDGET_REALIZED (widget))
+    return;
+  
   if (!GTK_WIDGET_NO_WINDOW (widget) && widget->parent)
     {
       int x, y;
@@ -5114,6 +5118,8 @@ gtk_widget_set_child_visible (GtkWidget *widget,
   g_return_if_fail (GTK_IS_WIDGET (widget));
   g_return_if_fail (!GTK_WIDGET_TOPLEVEL (widget));
 
+  g_object_ref (widget);
+
   if (is_visible)
     GTK_PRIVATE_SET_FLAG (widget, GTK_CHILD_VISIBLE);
   else
@@ -5136,6 +5142,8 @@ gtk_widget_set_child_visible (GtkWidget *widget,
       else
        gtk_widget_unmap (widget);
     }
+
+  g_object_unref (widget);
 }
 
 /**

diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
index 2fb60ba450c8423b1ec303d9fd0a274c26c5ac57..0e3c5f3db1cc0921a87bc039f9be08bd1e8ee8be 100644 (file)
--- a/gtk/gtkwindow.c
+++ b/gtk/gtkwindow.c
@@ -4562,6 +4562,9 @@ _gtk_window_unset_focus_and_default (GtkWindow *window,
 
 {
   GtkWidget *child;
+
+  g_object_ref (window);
+  g_object_ref (widget);
       
   if (GTK_CONTAINER (widget->parent)->focus_child == widget)
     {
@@ -4581,6 +4584,9 @@ _gtk_window_unset_focus_and_default (GtkWindow *window,
       
   if (child == widget)
     gtk_window_set_default (window, NULL);
+  
+  g_object_unref (widget);
+  g_object_unref (window);
 }
 
 /*********************************